Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CAMS Regional Air Quality Inventory #69

Merged
merged 14 commits into from
Nov 21, 2024
Merged

CAMS Regional Air Quality Inventory #69

merged 14 commits into from
Nov 21, 2024

Conversation

corink21
Copy link
Collaborator

@corink21 corink21 commented Nov 18, 2024

Script to read in regional air quality inventory from CAMS (v6.1-Ref2, https://eccad.sedoo.fr/#/metadata/608) on regular lat lon grid.


📚 Documentation preview 📚: https://emiproc--69.org.readthedocs.build/en/69/

emiproc/grids.py Outdated Show resolved Hide resolved
emiproc/inventories/cams_reg_aq.py Show resolved Hide resolved
emiproc/inventories/cams_reg_aq.py Outdated Show resolved Hide resolved
@lionel42
Copy link
Collaborator

@corink21 If you can change the few points I mentioned above, that would be perfect

@lionel42
Copy link
Collaborator

@corink21 I tried to download the the files and run the processing. Which worked well.

I have 2 more comments.

  • The temporal profiles don't originate from the website you have put right ? In this case would you agree to remove the temporal profiles part of the code and put it somewhere else in you processing script ?

  • Could you check the units ? emiproc uses kg/year/cell

@corink21
Copy link
Collaborator Author

  • The temporal profiles don't originate from the website you have put right ? In this case would you agree to remove the temporal profiles part of the code and put it somewhere else in you processing script ?

I agree that one should move the profiles part somewhere else. I am using the TNO profiles, which don't originate from the ECCAD website.

  • Could you check the units ? emiproc uses kg/year/cell

The emission dataset uses Tg/year/cell, which is then converted to kg/year/cell with UNIT_CONVERSION_FACTOR = 1.e9.

@lionel42 lionel42 merged commit 1fc797d into master Nov 21, 2024
4 checks passed
@corink21 corink21 deleted the cams_aq_inv branch November 21, 2024 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants